!Hotfix: Swagger 인증 표시를 정리하고 공개 엔드포인트를 분리한다 #73
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 Why - 해결하려는 문제가 무엇인가요?
✅ What - 무엇이 변경됐나요?
bearerAuth로 통일@SecurityRequirement(name = "bearerAuth")적용security = {}명시@OpenAPIDefinition.security제거하여 불필요한 자물쇠 표시/강제 보안 적용 제거""반환 →null반환으로 수정하여 정상 토큰이 블랙리스트로 판정되는 문제 해결🛠️ How - 어떻게 해결했나요?
Swagger 인증 문제 해결
bearerAuth로 통일하고,@SecurityRequirement(name = "bearerAuth")를 명시했습니다.@OpenAPIDefinition.security를 제거해, 모든 API에 보안이 강제 적용되는 문제를 줄였습니다./v1/experts,/v1/experts/{expertId})는 Swagger에서 보안 표시가 붙지 않도록security = {}로 명시했습니다.블랙리스트 오판(로그아웃 토큰 체크) 해결
""를 반환하던 동작 때문에,블랙리스트 체크가 항상 true로 평가될 수 있어
null반환으로 수정했습니다.고민/예외 사항
💬 기타 코멘트
@SecurityRequirement의 name 불일치가 원인이었습니다.curl로/v1/members호출하여 인증 성공 확인Summary by CodeRabbit
릴리스 노트
새로운 기능
버그 수정
개선 사항
✏️ Tip: You can customize this high-level summary in your review settings.